Skip to content

Remove stray backslash and some nits for "Multiple Base Classes" topic #5804

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 4 commits into from
Oct 2, 2025
Merged

Remove stray backslash and some nits for "Multiple Base Classes" topic #5804

merged 4 commits into from
Oct 2, 2025

Conversation

Rageking8
Copy link
Contributor

Perform some cleanup/nits and remove stray backslash (introduced in commit 79cc5de) that renders verbatim:

image

@prmerger-automator PRMerger Automator
Copy link
Contributor

@Rageking8 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit 2c56428:

✅ Validation status: passed

File Status Preview URL Details
docs/cpp/multiple-base-classes.md ✅Succeeded

For more details, please refer to the build report.

Copilot
Copilot AI reviewed Sep 26, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR cleans up the "Multiple Base Classes" documentation by fixing a rendering issue and standardizing formatting. The main purpose is to remove a stray backslash that was incorrectly rendering in the documentation and to apply consistent formatting improvements throughout the file.

Key changes:

  • Remove stray backslash from pragma directive that was rendering verbatim
  • Standardize image source paths and code formatting
  • Update relative link paths for consistency

@ShannonLeavitt
Copy link
Contributor

@TylerMSFT

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Sep 26, 2025
@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit e397c59:

✅ Validation status: passed

File Status Preview URL Details
docs/cpp/multiple-base-classes.md ✅Succeeded

For more details, please refer to the build report.

Rageking8
Rageking8 commented Sep 27, 2025
View reviewed changes
@TylerMSFT
Copy link
Collaborator

#sign-off

@v-dirichards v-dirichards merged commit 7180e12 into MicrosoftDocs:main Oct 2, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@TylerMSFT TylerMSFT TylerMSFT approved these changes

Assignees

@TylerMSFT TylerMSFT

Labels
aq-pr-triaged Tracking label for the PR review team Change sent to author cpp-lang/subsvc ready-to-merge visual-cpp/svc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Rageking8 @ShannonLeavitt @TylerMSFT @v-dirichards